Skip to content

CMake: Set GCC version specific subfolder for MinGW #4671

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

CMake: Set GCC version specific subfolder for MinGW #4671

wants to merge 1 commit into from

Conversation

beutlich
Copy link
Member

@beutlich beutlich commented Jun 23, 2025 •
edited
Loading

GCC x.y.z will create a subfolder gccxyz.

@beutlich beutlich added this to the MSL4.2.0 milestone Jun 23, 2025
@beutlich beutlich requested review from maltelenz and HansOlsson June 23, 2025 19:46
@beutlich beutlich added L: Resources Issue addresses Modelica/Resources (excl. C-Sources) V: 4.2.0-dev Issue originates in MSL v4.2.0-dev (and is not present in earlier releases) labels Jun 23, 2025
maltelenz
maltelenz reviewed Jul 14, 2025
View reviewed changes
Copy link
Contributor

@maltelenz maltelenz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could/should it use CMAKE_C_COMPILER_VERSION?

The general idea seems fine, but System Modeler doesn't have MinGW support, so I cannot do any real testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maltelenz maltelenz maltelenz left review comments

@HansOlsson HansOlsson Awaiting requested review from HansOlsson

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
L: Resources Issue addresses Modelica/Resources (excl. C-Sources) V: 4.2.0-dev Issue originates in MSL v4.2.0-dev (and is not present in earlier releases)
Projects
None yet
Milestone
MSL4.2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@beutlich @maltelenz